Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Check Availability of gICS and gPAS During Startup #630

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Check Availability of gICS and gPAS During Startup #630

wants to merge 2 commits into from
+623 −156

Conversation

trobanga
Copy link
Contributor

No description provided.

@trobanga trobanga self-assigned this Feb 25, 2025
@trobanga trobanga linked an issue Feb 25, 2025 that may be closed by this pull request
Check Availability of gICS and gPAS During Startup #626
Open
@trobanga trobanga requested a review from knoppiks February 25, 2025 10:04
@trobanga trobanga force-pushed the 626-check-availability-of-gics-and-gpas-during-startup branch from 9593d19 to 94b6b56 Compare February 25, 2025 10:05
@trobanga trobanga marked this pull request as draft February 25, 2025 10:05
@codecov Codecov
Copy link

codecov bot commented Feb 25, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 93.20388% with 7 lines in your changes missing coverage. Please review.

Project coverage is 96.22%. Comparing base (b46e1ec) to head (73a8082).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...a/consent/configuration/GicsFhirConfiguration.java 75.00% 4 Missing ⚠️
...uration/GpasFhirDeIdentificationConfiguration.java 78.57% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #630      +/-   ##
============================================
- Coverage     96.40%   96.22%   -0.19%     
- Complexity      419      431      +12     
============================================
  Files           112      117       +5     
  Lines          1614     1694      +80     
  Branches         43       45       +2     
============================================
+ Hits           1556     1630      +74     
- Misses           42       49       +7     
+ Partials         16       15       -1
Files with missing lines Coverage Δ
...ava/care/smith/fts/cda/impl/TCACohortSelector.java 100.00% <100.00%> (ø)
...tca/consent/GicsFhirConsentedPatientsProvider.java 100.00% <100.00%> (ø)
...are/smith/fts/tca/deidentification/GpasClient.java 100.00% <100.00%> (+3.03%) ⬆️
...ava/care/smith/fts/tca/rest/ConsentController.java 100.00% <100.00%> (ø)
...smith/fts/tca/rest/DeIdentificationController.java 100.00% <ø> (ø)
...main/java/care/smith/fts/util/FhirClientUtils.java 100.00% <100.00%> (ø)
...l/src/main/java/care/smith/fts/util/FhirUtils.java 100.00% <ø> (ø)
...main/java/care/smith/fts/util/RetryStrategies.java 92.30% <100.00%> (+4.80%) ⬆️
...mith/fts/util/error/fhir/FhirConnectException.java 100.00% <100.00%> (ø)
...ith/fts/util/error/fhir/FhirErrorResponseUtil.java 100.00% <100.00%> (ø)
... and 5 more

@trobanga trobanga force-pushed the 626-check-availability-of-gics-and-gpas-during-startup branch 15 times, most recently from 332682a to 96f2663 Compare February 28, 2025 10:46
@trobanga trobanga marked this pull request as ready for review February 28, 2025 10:47
@trobanga trobanga force-pushed the 626-check-availability-of-gics-and-gpas-during-startup branch from 96f2663 to 717f5bb Compare February 28, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@knoppiks knoppiks Awaiting requested review from knoppiks

Assignees

@trobanga trobanga

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Check Availability of gICS and gPAS During Startup
1 participant
@trobanga